Codecov Report

Merging #1555 into master will decrease coverage by 0.32%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1555      +/-   ##
==========================================
- Coverage   20.46%   20.14%   -0.33%     
==========================================
  Files         241      244       +3     
  Lines        5243     5327      +84     
  Branches      649      660      +11     
==========================================
  Hits         1073     1073              
- Misses       3678     3744      +66     
- Partials      492      510      +18

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e3bcd4d...73ac57e. Read the comment docs.

This looks good to me. I personally dislike having to "go backwards" but I understand the need.

I just realized I hand't audited other components. Verifying StatusBar is supported in 0.27

StatusBar is supported in 0.27. I think we're all good here.

I should pull this into my 0.27 project and verify that it all works as expected prior to merging.

Testing RN 0.27 right now. Will report back shortly

Have some minWidth and maxWidth problems:

WIP

Ran into two additional backward incompatibilities. For ease of use moving forward, I added a polyfills/ directory to address these head on.

StyleSheet.absoluteFillObject

This is only supported in recent versions. It's simply a preset style for a full screen absolutely positioned element.

MinMaxViewPolyfill

The min/max width/height features of styles are not available on earlier versions of React Native. This adds a Higher Order Component that can declaratively (through props) apply these bounding rules to any child component.

38ef3f6 moves the polyfill implementations into react-native-compat

👍

How far back do we want to support? I understand we want to make sure that we support older versions of React Native but there has to be a limit to how far back we go.

How far back do we want to support? I understand we want to make sure that we support older versions of React Native but there has to be a limit to how far back we go.

It's all about finding the proper balance. Storybook itself doesn't require much UI, so with minimal effort we're able to keep folks on earlier versions supported. If there's any drawback, I think it'll be on the contributor and reviewer side to keep those requirements communicated and enforced.

We currently support >=0.27, so for this release at the very least, we want to make sure we continue to satisfy that requirement without regressions.

I don't think this is the proper thread to make a formal decision, but it is something to consider moving forward with the compatibility support in, and observe how things go in future reviews.